Conversation
Bumps [gson](https://github.com/google/gson) from 2.8.5 to 2.8.9. - [Release notes](https://github.com/google/gson/releases) - [Changelog](https://github.com/google/gson/blob/master/CHANGELOG.md) - [Commits](google/gson@gson-parent-2.8.5...gson-parent-2.8.9) --- updated-dependencies: - dependency-name: com.google.code.gson:gson dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [jackson-databind](https://github.com/FasterXML/jackson) from 2.11.3 to 2.12.7.1. - [Release notes](https://github.com/FasterXML/jackson/releases) - [Commits](https://github.com/FasterXML/jackson/commits) --- updated-dependencies: - dependency-name: com.fasterxml.jackson.core:jackson-databind dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
ChunkReaderWriter now deletes existing chunk files if a write instruction is received that only contains fill values.
Update to new repository
…th_zarr_devs # Conflicts: # pom.xml
Bumps [org.assertj:assertj-core](https://github.com/assertj/assertj) from 3.19.0 to 3.27.7. - [Release notes](https://github.com/assertj/assertj/releases) - [Commits](assertj/assertj@assertj-core-3.19.0...assertj-build-3.27.7) --- updated-dependencies: - dependency-name: org.assertj:assertj-core dependency-version: 3.27.7 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com>
…on-gson-2.8.9 Bump gson from 2.8.5 to 2.8.9
…son.core-jackson-databind-2.12.7.1 Bump jackson-databind from 2.11.3 to 2.12.7.1
Fix for issue #39 when path contains spaces
…j-core-3.27.7 Bump org.assertj:assertj-core from 3.19.0 to 3.27.7
# Conflicts: # pom.xml
|
|
||
| <groupId>dev.zarr</groupId> | ||
| <artifactId>jzarr</artifactId> | ||
| <version>0.4.3-SNAPSHOT</version> |
There was a problem hiding this comment.
Is there a reason to change this version number and decrease the minor version?
There was a problem hiding this comment.
Dear Sébastien
No reason. Just a mistake.
Feel free to correct the version number.
Thank you very much for pointing that out!
Best Regards
Sabine
There was a problem hiding this comment.
👍 converted into a suggestion to restore the current version number
Co-authored-by: Sébastien Besson <sbesson@glencoesoftware.com>
joshmoore
left a comment
joshmoore
left a comment
There was a problem hiding this comment.
At a fairly high-level, all the changes look fine to me. It might be useful to have a list of links (or fix #NNN) style comments on the PR to know how this updates the wider ecosystem.
Once everyone's happy, 👍 on a release (I'd suggest 0.5.0, personally). If anything goes wrong in getting that to Maven Central, please let me know and I'll look into it. (As a follow-on, we should likely get more people set up with managing jzarr on Maven Central.)
6 participants
Merging the diverged branches between bcdev/jzarr and zarr-developers/jzarr .
Mostly, we need to get the settings in pom.xml right
This PR also resolves #24 and introduces functionality to prevent that empty (fill value-only) chunks are written.